Skip to content

[wip] feat: integrate new revm#14115

Merged
klkvr merged 15 commits intoklkvr/alloy-evmfrom
klkvr/new-revm
Feb 17, 2025
Merged

[wip] feat: integrate new revm#14115
klkvr merged 15 commits intoklkvr/alloy-evmfrom
klkvr/new-revm

Conversation

@klkvr
Copy link
Member

@klkvr klkvr commented Jan 30, 2025

Based on #14021

Updates revm to latest

@emhane emhane added A-execution Related to the Execution and EVM A-dependencies Pull requests or issues that are about dependencies labels Feb 4, 2025
@klkvr klkvr force-pushed the klkvr/alloy-evm branch 4 times, most recently from e5049d7 to caef4ba Compare February 7, 2025 00:49
@klkvr klkvr force-pushed the klkvr/new-revm branch 2 times, most recently from a1c7307 to 4619a86 Compare February 12, 2025 17:57
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's do this

Comment on lines 753 to 754
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should reconsider this now and remove the fill function and instead do -> TxEnv

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, why does this exist?

Comment on lines 98 to 102
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is so much worse now imo -.-

klkvr and others added 3 commits February 17, 2025 21:28
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Co-authored-by: Arsenii Kulikov <klkvrr@gmail.com>
@klkvr klkvr marked this pull request as ready for review February 17, 2025 19:35
@klkvr klkvr merged commit 9bf7709 into klkvr/alloy-evm Feb 17, 2025
36 checks passed
@klkvr klkvr deleted the klkvr/new-revm branch February 17, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-dependencies Pull requests or issues that are about dependencies A-execution Related to the Execution and EVM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants